Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add conda-based Continuous Integration and fix compilation with Gazebo on Windows installed via conda #33

Merged
merged 3 commits into from
Apr 23, 2021

Conversation

traversaro
Copy link
Member

@traversaro traversaro commented Apr 22, 2021

Add a simple conda-based CI. The goal is to easily test on Windows/macOS/Linux with all dependencies enabled. This complements the existing CI that instead used apt/homebrew/vcpkg (and compiles robotology dependencies from source).

Furthermore, fix compilation of the library that uses Gazebo in Windows by adding the line:

target_link_directories(${PROJECT_NAME}-world PRIVATE ${GAZEBO_LIBRARY_DIRS})

In general target_link_directories and link_directories commands should not be used in CMake (see the note at the end of https://cmake.org/cmake/help/latest/command/target_link_directories.html), but when linking Gazebo (and Gazebo_LIBRARIES) this is unfortunately necessary due to many libraries that are provided without full paths, see http://gazebosim.org/tutorials/?tut=plugins_hello_world .

This fixes issue robotology/robotology-superbuild#693 .

Similar PRs:

@traversaro
Copy link
Member Author

traversaro commented Apr 22, 2021

The main goal is to reproduce and fix robotology/robotology-superbuild#693 this was reproduced and fixed.

@traversaro traversaro changed the title Add conda-based Continuous Integration [WIP] Add conda-based Continuous Integration Apr 22, 2021
@traversaro
Copy link
Member Author

WIP as I wanted to check if I can reproduce robotology/robotology-superbuild#693 and fix it before merging.

@traversaro
Copy link
Member Author

Error reproduced:

2021-04-22T15:41:56.5633232Z LINK : fatal error LNK1181: cannot open input file 'OgreBites.lib' [D:\a\icub-basic-demos\icub-basic-demos\build\demoRedBall\demoRedBall-world.vcxproj]
2021-04-22T15:41:56.6591852Z   Building Custom Rule D:/a/icub-basic-demos/icub-basic-demos/demoYoga/CMakeLists.txt

@traversaro traversaro changed the title [WIP] Add conda-based Continuous Integration Add conda-based Continuous Integration Apr 22, 2021
@traversaro traversaro requested a review from Nicogene April 22, 2021 16:10
@traversaro
Copy link
Member Author

@Nicogene the PR is now ready for review.

@traversaro traversaro changed the title Add conda-based Continuous Integration Add conda-based Continuous Integration and fix compilation with Gazebo on Windows installed via conda Apr 22, 2021
@traversaro
Copy link
Member Author

fyi @Tobias-Fischer, you may be happy to know that the good old red ball demo is getting a Conda-based CI. : )

@Tobias-Fischer
Copy link
Member

Tobias-Fischer commented Apr 22, 2021

How cool is that? Very nice, good work @traversaro :)
Amazing just to see this line and how much pain it will save dozens of people in the future: mamba install -c robotology yarp icub-main gazebo icub-contrib-common qt opencv

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants